-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(issue-details): Add analytics about sidebar/sections #80766
Conversation
|
||
return { | ||
sidebar_open: localStorage.getItem('issue-details-sidebar-open') === 'true', | ||
...sidebarOpenSections, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doing it this way because we can't really send an array over and if we sent an individual event for each section open it would be too many events
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
doesn't feel like this will be valuable to me since it also depends on if the page even has those sections
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh its just the sidebar
@@ -7,7 +7,7 @@ import { | |||
useReducer, | |||
} from 'react'; | |||
|
|||
export const enum SectionKey { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how are these in the sidebar?
decided that the existing analytics events should be enough |
this pr updates the
issue_details.viewed
event with info about if the sidebar was open and what fold sections are open.